-
Notifications
You must be signed in to change notification settings - Fork 110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AAP-11261] Add MQTT event source #113
Conversation
Adding an MQTT event source for rulebooks - Tested
Linting
hi team, any news on this one? |
@kubealex
|
Thank you @kurokobo! I just merged your contribution into the branch 🚀 |
React to decommissioning of asyncio_mqtt
I am very much looking forward to seeing this merged. :) |
I've been using this patch locally and has been working as expected, thanks!! |
Hi @kubealex We want to express our sincere appreciation for your effort and contribution. After careful consideration, we have concluded that this plugin doesn't align with the scope of our official collection, and unfortunately, it will not be included. However, this does not diminish the value of your work. We believe your plugin can still be highly beneficial and encourage you to offer it through your own collection. You are welcome to upload it to galaxy with the "eda" tag. As a fellow Red Hatter, we also suggest reaching out to your product management team to explore if such a collection could be supported within Red Hat as a maintained collection. Thank you for your understanding and contribution. |
Thank you for the feedback Alex, much appreciated! @jpmens @tonyskapunk and anyone who could be interested in using or including it, no worries, I'll leave it here [1] the link to the custom collection that includes the plugin. |
@Alex-Izquierdo In the documentation, the
This is why I reached out here to find the status of MQTT plugin. This documentation might cause users misunderstanding that an officially maintained MQTT plugin exists (yes, I am one of the users who misunderstood). It would be great if the MQTT plugin is officially implemented as the documentation shows, but since as you stated this plugin no longer merged into this collection, the documentation should be updated. Please let me know if I should create an issue in ansible-rulebook or create a PR to remove MQTT from the documentation. Thanks in advance! |
Thanks for raising this issue @kurokobo |
Thanks for rapid update😃 |
@kubealex thanks for the link to the custom collection that includes the plugin (https://galaxy.ansible.com/kubealex/eda), but that is 404 which, I assume, is because of the transition from Galaxy to -NG. The current link is https://galaxy.ansible.com/ui/repo/published/kubealex/eda/ but I think it's probably safer to mention the link to your repository at https://github.com/kubealex/kubealex.eda |
Adding an MQTT event source for rulebooks - Tested